Skip to content

Replace bitnami Helm chart dependencies with upstream alternatives#593

Draft
Copilot wants to merge 5 commits intochore/bitnamifrom
copilot/sub-pr-553
Draft

Replace bitnami Helm chart dependencies with upstream alternatives#593
Copilot wants to merge 5 commits intochore/bitnamifrom
copilot/sub-pr-553

Conversation

Copy link

Copilot AI commented Feb 5, 2026 •
edited
Loading

Bitnami dependencies are deprecated. This PR replaces all three bitnami chart dependencies with stable alternatives using custom templates and official upstream images.

Changes

PostgreSQL

  • Already replaced with CloudNativePG operator in custom template

Keycloak

  • Created keycloak-service.yaml for service definition
  • Fixed configmap.yaml to use .AsConfig directly without nested key
  • Fixed keycloak-deployment.yaml:
    • PostgreSQL sidecar now uses TCP probes instead of HTTP
    • Changed port protocol from UDP to TCP
    • Added conditional for optional extraEnvVars
  • Updated ingress.yaml to reference {{ include "api-platform.fullname" . }}-keycloak instead of bitnami helper template

external-dns

  • Created external-dns-deployment.yaml with:
    • Deployment using registry.k8s.io/external-dns/external-dns:v0.15.0
    • Full RBAC (ServiceAccount, ClusterRole, ClusterRoleBinding)
    • Cloudflare API token secret support
  • Removed dependency from Chart.yaml
  • Updated values.yaml with official image repository

Template Structure

The keycloak realm ConfigMap was incorrectly nesting data:

# Before (invalid)
data:
  realm.json: |
    {{ (.Files.Glob .Values.keycloak.importRealm.path).AsConfig | indent 2 }}
    # This created: realm.json: realm-demo.json: { ... }

# After (valid)
data:
{{ (.Files.Glob .Values.keycloak.importRealm.path).AsConfig | indent 2 }}
# This creates: realm-demo.json: |

Chart now passes helm lint and renders successfully.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Feb 5, 2026
Draft
3 tasks
Copy link
Author

Copilot AI commented Feb 5, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • artifacthub.io
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)
  • kubernetes-sigs.github.io
    • Triggering command: /usr/local/bin/helm helm dependency update (dns block)
    • Triggering command: /usr/local/bin/helm helm repo add external-dns REDACTED (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Fix invalid Helm chart and replace external DNS dependency Replace bitnami Helm chart dependencies with upstream alternatives Feb 5, 2026
Copilot AI requested a review from vincentchalamon February 5, 2026 15:30
Copilot AI and others added 4 commits February 5, 2026 16:47
@vincentchalamon
Initial plan
5c6a71e
@vincentchalamon
fix: replace bitnami/external-dns and fix Helm template issues
58124cb 
Co-authored-by: vincentchalamon <407859+vincentchalamon@users.noreply.github.com>
@vincentchalamon
fix: resolve Helm chart YAML parsing issues
cf9fb03 
Co-authored-by: vincentchalamon <407859+vincentchalamon@users.noreply.github.com>
@vincentchalamon
fix: correct PostgreSQL sidecar container probes and port protocol
d98e96f 
Co-authored-by: vincentchalamon <407859+vincentchalamon@users.noreply.github.com>
@vincentchalamon vincentchalamon added the deploy Deploys Pull Request label Feb 5, 2026
slim
slim suggested changes Feb 17, 2026
View reviewed changes
helm/api-platform/templates/external-dns-deployment.yaml
Copy link
Contributor

@slim slim Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot why not use the chart from here https://kubernetes-sigs.github.io/external-dns/latest/charts/external-dns/#maintainers as a dependency ?

Copy link
Contributor

@slim slim Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vincentchalamon je crois que copilot me calcule pas parce que j'ai pas d'accès en écriture sur le repo. Tu pourrais lui dire toi ? :-D

Copy link
Contributor

@vincentchalamon vincentchalamon Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vais tenter, mais je crois que Marion m'a désactivé mon accès Copilot au profit de Claude.

Copy link
Contributor

@vincentchalamon vincentchalamon Feb 17, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot why not use the chart from here kubernetes-sigs.github.io/external-dns/latest/charts/external-dns#maintainers as a dependency ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vincentchalamon vincentchalamon Awaiting requested review from vincentchalamon

1 more reviewer

@slim slim slim requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@vincentchalamon vincentchalamon

Copilot code review Copilot

Labels

deploy Deploys Pull Request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@slim @vincentchalamon